[test] Remove -emit-llvm from the and-errors.hlsl test to avoid writing to a potentially write-protected directory
#128047
Conversation
llvmbot
added
clang
|
@llvm/pr-subscribers-hlsl Author: Deric Cheung (Icohedron) Changes@mikaelholmen mentioned that the This PR removes the Full diff: https://github.com/llvm/llvm-project/pull/128047.diff 1 Files Affected:
diff --git a/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl b/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
index 0a99feb023d73..d9721140b9bf9 100644
--- a/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
+++ b/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
@@ -1,6 +1,5 @@
// RUN: %clang_cc1 -finclude-default-header -triple \
-// RUN: dxil-pc-shadermodel6.3-library %s \
-// RUN: -emit-llvm -O1 -verify
+// RUN: dxil-pc-shadermodel6.3-library %s -O1 -verify
bool test_too_few_arg(bool a) {
return __builtin_hlsl_and(a);
|
|
@llvm/pr-subscribers-clang Author: Deric Cheung (Icohedron) Changes@mikaelholmen mentioned that the This PR removes the Full diff: https://github.com/llvm/llvm-project/pull/128047.diff 1 Files Affected:
diff --git a/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl b/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
index 0a99feb023d73..d9721140b9bf9 100644
--- a/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
+++ b/clang/test/SemaHLSL/BuiltIns/and-errors.hlsl
@@ -1,6 +1,5 @@
// RUN: %clang_cc1 -finclude-default-header -triple \
-// RUN: dxil-pc-shadermodel6.3-library %s \
-// RUN: -emit-llvm -O1 -verify
+// RUN: dxil-pc-shadermodel6.3-library %s -O1 -verify
bool test_too_few_arg(bool a) {
return __builtin_hlsl_and(a);
|
|
Thank you @Icohedron ! |
6 participants
@mikaelholmen mentioned that the
-emit-llvmargument isn't necessary for theand-errors.hlsltest and may cause issues due to writing to the current (potentially write-protected) directory.This PR removes the
-emit-llvmargument from clang in the RUN lines of the test.